Skip to content

Convert typed array to 'regular' arrays before sendToCloud#2995

Merged
etpinard merged 1 commit intomasterfrom
graphJson-for-typed-arrays
Sep 11, 2018
Merged

Convert typed array to 'regular' arrays before sendToCloud#2995
etpinard merged 1 commit intomasterfrom
graphJson-for-typed-arrays

Conversation

@etpinard
Copy link
Contributor

  • as JSON.stringify(new Float32Array([1,2,3])) gives: "{"0":1,"1":2,"2":3}"
  • use Lib.simpleMap as opposed to Array.prototype.slice.call(d) to avoid potential 'exceed stack size' errors

peek 2018-09-11 12-07

cc @alexcjohnson

- as JSON.stringify(new Float32Array([1,2,3])) gives:
  "{"0":1,"1":2,"2":3}"
- use Lib.simpleMap as opposed to Array.prototype.slice.call(d)
  to avoid potential 'exceed stack size' errors
@etpinard etpinard added bug something broken status: reviewable labels Sep 11, 2018
@alexcjohnson
Copy link
Collaborator

Beautiful! 💃

@etpinard etpinard merged commit d0ee187 into master Sep 11, 2018
@etpinard etpinard deleted the graphJson-for-typed-arrays branch September 11, 2018 16:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug something broken

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants